-
Notifications
You must be signed in to change notification settings - Fork 145
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Adding .fog.example and amending docs #207
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for contributing!
See my comments inline.
README.md
Outdated
@@ -76,6 +76,8 @@ my_credential: | |||
google_json_key_location: /path/to/my-project-xxxxxxxxxxxx.json | |||
``` | |||
|
|||
Or simply `cat .fog.example >> ~.fog` and edit parameters in the file. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
~.fog
is an incorrect glob, as it'll resolve to: /Users/myuser.fog, you need a slash: ~/.fog
.fog.example
Outdated
@@ -0,0 +1,12 @@ | |||
##################################################### | |||
# This is configuration snippet to configure Google Cloud Platform | |||
# for fig library. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
typo
.fog.example
Outdated
# 2. Follow instructions to generate a private key: | ||
# https://cloud.google.com/storage/docs/authentication#generating-a-private-key | ||
|
||
my_credential: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Although I do admit that's our doc issue, I'm not sure if my_credential
will actually work by default if you didn't explicitly load them via Fog::Credentials. AFAIK only "default" works by default and whatever is in "test" is loaded by tests.
Have you tested this?
ACK on comments - thanks.
|
New commit which addresses Temikus comments: palladius@93781eb |
@palladius LGTM, thanks for contributing \o/ Sorry it took a while. One last note - I suggest setting your git config to a proper email, so you get the contribution tracked on github properly. Currently your commit's email is set to "[email protected]". Either associate that with GitHub or:
Cheers! |
(You'll need to |
Thanks bros! Done the following: git rebase -i HEAD~2 |
Thanks! |
[trying my first pull request here! So apologies for newbbisities]
Adding a sample file for .fog and proposing in the docs to just append this sample into your home fog (I'm not an expert but I presume this is safer than a CP as people might have stuff already in there). Ideally if you like this idea I can actually remove the same stuff from README (more DRY) - but I wasnt sure.